Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to oclif core v3 #102

Closed
wants to merge 11 commits into from
Closed

Upgrade to oclif core v3 #102

wants to merge 11 commits into from

Conversation

ryandagg
Copy link
Contributor

@ryandagg ryandagg commented Oct 13, 2023

https://gus.lightning.force.com/lightning/r/ADM_Work__c/a07EE00001cOMR2YAO/view
Had to update the branch protection rules for master to accommodate the new test names.
https://github.com/heroku/heroku-cli-command/settings/branch_protection_rules/1052008
From
test ($OS-matix, lts/gallium) && test ($OS-matix, lts/fermium)
to
test ($OS-matix, 18.x)

@ryandagg ryandagg requested a review from a team as a code owner October 13, 2023 20:18
src/request-id.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@k80bowman k80bowman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approved because this looks good, but it looks like you will also need to update the node versions we are running our CI against.

Copy link

@lancedikson lancedikson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great stuff! I just tested this branch on my new heroku cli plugin, and it works great. The latest oclif generate scaffolds a structure for the oclif3-compatible plugins only, and the latest stable heroku-cli/command gave me a types mismatch error in TypeScript. Any chances this PR gets merged any time soon? :) I'd really like to release my plugin, though it would be tricky if one of it's dependencies is pulled from Gh repo and should be built right after installing.

@lancedikson
Copy link

Hi @ryandagg! Any chances for this to be merged soon? :)

@eablack
Copy link
Contributor

eablack commented Apr 11, 2024

Hi @lancedikson I can see you are excited about this (we are too!).. However, there is some other significant work that must be done (primarily with our own plugins) before we can incorporate this into the CLI. The good news is that work has been prioritized and we will be tackling it soon. For now, I'm going to close this PR.

@eablack eablack closed this Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants